Skip to content

fix(moshscript): skip sleep during dry runs#32

Merged
ralyodio merged 1 commit into
moshcoder:mainfrom
padsbanger:fix/dry-run-sleep
Jul 25, 2026
Merged

fix(moshscript): skip sleep during dry runs#32
ralyodio merged 1 commit into
moshcoder:mainfrom
padsbanger:fix/dry-run-sleep

Conversation

@padsbanger

Copy link
Copy Markdown
Contributor

Summary

  • make sleep(ms) honor the moshscript dry-run contract
  • narrate the intended delay instead of blocking
  • add a regression test that verifies Atomics.wait is not called

This fixes the PRD 0004 R7 dry-run guarantee: previews now avoid the sleep side effect along with process and network effects.

Verification

  • node --test test/commands.test.mjs (13/13 passed)
  • npm test on Node 22 (135/135 passed)

@ralyodio
ralyodio merged commit 76c3047 into moshcoder:main Jul 25, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants